-
Notifications
You must be signed in to change notification settings - Fork 10
ComposerUnused: use composer version instead of phar #58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
fixed the Dependency Tests, |
A tip here from me. Use this tool instead: https://github.com/shipmonk-rnd/composer-dependency-analyser. It's absolutely wonderful compared to composer require checker and composer unused :-) |
I didn't want to switch to a new tool instead, i wanted to fix the existing setting first. i can do that later, first i need to fix the |
I used the new dependency analyzer (i will remove the old code soon) and fixed the version specific dependencies. just the Psalm problem with TypeResolver and the new Type-Info |
@loevgaard do you have any idea who to fix the Symfony Version Specific psalm problem? (7.3+ has TypeInfo Component) i made a Issue ticket because i couldn't solve it myself: vimeo/psalm#11560 |
@loevgaard I fixed the psalm problems with Can this be Squash Merged, so we can look into DoctrineBundle 3.0 and PHP 8.4? |
I will make an extra MR later to update the PHPUnit version to 12+ i needed to rename some tests methods to remove |
@loevgaard this uses Composer Package instead of Phar to prevent:
Your Composer dependencies require a PHP version ">= 8.2.0"